-
Notifications
You must be signed in to change notification settings - Fork 12.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch x64 windows msvc target to use lld
by default.
#79448
Conversation
r? @estebank (rust-highfive has picked a reviewer for you, use r? to override) |
Should this be added to 1.50? I believe due to the many stabilizations it will be a base MSRV for a long time. Maybe LLD causes regressions/bugs so maybe it will be switched back in a future release. If 1.50.0 included lld though one would have to deal with it for a long time. |
As an alternative, one could add it, but make it conditional on the nightly channel. The stable and beta releases would still use the default setting. |
Is it still x86_64_pc_windows_msvc or should it be x86_64_pc_windows_none or something? |
You still need MSVC toolchain for it to build, target vendor shouldn't be changed here. |
There is a proposal for a pure rust target in the todo list in #58713 |
@crlf0710 |
Thanks for the review! Unfortunately this seems to need a larger change than i thought, and i'm not very confident on getting the cross-compilation stuff correct... I'll just close this PR before i sort it out. Everybody is welcome to pick this up. Related zulip stream: https://rust-lang.zulipchat.com/#narrow/stream/131828-t-compiler/topic/unblocked.3A.20using.20lld.20by.20default.20for.20x64.20msvc.20windows |
cc #71520 .